Skip to content

refactor: split inspect commands from sync runtime#16

Merged
anandpant merged 1 commit into
mainfrom
feat/inspect-binary-split
Apr 20, 2026
Merged

refactor: split inspect commands from sync runtime#16
anandpant merged 1 commit into
mainfrom
feat/inspect-binary-split

Conversation

@anandpant

@anandpant anandpant commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a repo-local convex-inspect binary for source inspection
  • narrow convex-sync down to the heavy export/runtime commands only
  • clean up the leftover tracked old crate paths from the first refactor

What Changed

  • add apps/convex-inspect with:
    • schemas
    • snapshot
    • deltas
  • narrow apps/convex-sync to:
    • sync-once
    • materialize-staging
    • publish-s3
    • run
  • update justfile so schemas / snapshot / deltas now route to convex-inspect
  • update root and app readmes so the two-binary shape is explicit
  • update release docs to keep convex-inspect repo-local and convex-sync as the only released binary
  • remove the leftover tracked crates/convex-cdc-core and crates/convex-target-s3 paths now that their renamed replacements are the real source of truth

Why

This lands the final binary boundary we aligned on, not an intermediate state:

  • convex-inspect is the lightweight source-inspection tool
  • convex-sync is the heavy parquet -> staging -> S3/export runtime

That makes the command surface clearer and sets up a later CI split between light and heavy lanes.

Verification

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo test --workspace
  • cargo run -p convex-inspect -- --help
  • cargo run -p convex-sync -- --help
  • actionlint -config-file .github/actionlint.yaml .github/workflows/*.yml

Open in Devin Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@anandpant
anandpant merged commit 4a63407 into main Apr 20, 2026
7 checks passed
@anandpant
anandpant deleted the feat/inspect-binary-split branch April 20, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant